Skip to content

fix: remove signature from store when delete returns 404#6036

Merged
vitormattos merged 1 commit intomainfrom
fix/remove-signature-from-store-on-404
Dec 9, 2025
Merged

fix: remove signature from store when delete returns 404#6036
vitormattos merged 1 commit intomainfrom
fix/remove-signature-from-store-on-404

Conversation

@vitormattos
Copy link
Copy Markdown
Member

When attempting to delete a signature that no longer exists on the server (404 response), the signature was not being removed from the Pinia store, leaving stale data in the UI.

This fix checks for 404 status in the error handler and clears the signature from the store, ensuring the UI state matches the server state even when the signature was already deleted.

Also improved error handling to safely access nested response properties and provide a fallback error message.

When attempting to delete a signature that no longer exists on the
server (404 response), the signature was not being removed from the
Pinia store, leaving stale data in the UI.

This fix checks for 404 status in the error handler and clears the
signature from the store, ensuring the UI state matches the server
state even when the signature was already deleted.

Also improved error handling to safely access nested response
properties and provide a fallback error message.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos added this to the Next Major (33) milestone Dec 9, 2025
@vitormattos vitormattos self-assigned this Dec 9, 2025
@github-project-automation github-project-automation Bot moved this to 0. Needs triage in Roadmap Dec 9, 2025
@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable32

@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable31

@vitormattos vitormattos merged commit 5f11c6f into main Dec 9, 2025
58 checks passed
@vitormattos vitormattos deleted the fix/remove-signature-from-store-on-404 branch December 9, 2025 01:13
@github-project-automation github-project-automation Bot moved this from 0. Needs triage to 4. to release in Roadmap Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant